Skip to content

add code cov#29

Open
Aurashk wants to merge 1 commit intomainfrom
add-code-cov
Open

add code cov#29
Aurashk wants to merge 1 commit intomainfrom
add-code-cov

Conversation

@Aurashk
Copy link
Contributor

@Aurashk Aurashk commented Mar 18, 2026

Description

Adds codecov action to the Linux job. Secret was also added to the repo

Close #8

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future.
    (Indicate issue here: # (issue))

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@Aurashk Aurashk marked this pull request as ready for review March 19, 2026 16:37
@Aurashk Aurashk requested review from alexdewar and Copilot March 19, 2026 16:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Codecov coverage reporting to the existing CI workflow to support issue #8 (test coverage tracking).

Changes:

  • Upload coverage.xml to Codecov on Linux CI runs using codecov/codecov-action@v5.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment on lines +117 to +123
- name: Upload coverage to Codecov
if: runner.os == 'Linux'
uses: codecov/codecov-action@v5
with:
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
fail_ci_if_error: true
with:
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
fail_ci_if_error: true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set up test coverage tracking with codecov

3 participants